Skip to content

Conversation

@h-east
Copy link
Member

@h-east h-east commented Feb 1, 2025

'diffopt' のところはアルファベット順にソートされましたが、"linematch:" が追加されただけです。

Copy link
Member

@mityu mityu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ぼちぼち復帰できそうです。

2箇所だけコメントしました。他は良さげです。

doc/options.jax Outdated
'scrollbind' がオンのときには大抵便利である。

foldcolumn:{n} 差分モードの開始時にオプション 'foldcolumn' を
{n}に設定する。
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

原文の "Without this 2 is used." の訳が抜けていそうです。...ただこの文が何を言ってるのかがよくわからない...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

よくわかりませんね... 2だけ設定できない?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あ、「これが設定されていなければ、デフォルト値として 2 が使用される」という意味ですかね?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

しっくりきました。

doc/options.jax Outdated
Comment on lines 2998 to 2999
行の2 バッファ差分ハンク、またはそれぞれ 20 行
の 3 バッファ差分ハンクの揃えが可能になる。
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
行の2 バッファ差分ハンク、またはそれぞれ 20
3 バッファ差分ハンクの揃えが可能になる。
行の 2 バッファ差分ハンク、またはそれぞれ 20
行の 3 バッファ差分ハンクの揃えが可能になる。

空白抜けです

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L2997の " の前に空白がないのも発見したのでローカルで修正しました。

@h-east h-east requested review from mityu and tsuyoshicho February 3, 2025 11:15
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

気付けたところはすべて指摘しました。

h-east and others added 7 commits February 3, 2025 21:47
@h-east h-east requested a review from tsuyoshicho February 3, 2025 12:59
Copy link
Contributor

@tsuyoshicho tsuyoshicho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mityu
Copy link
Member

mityu commented Feb 7, 2025

LGTM 👍

@h-east h-east merged commit c96bff7 into vim-jp:master Feb 7, 2025
2 checks passed
@h-east h-east deleted the update-options branch February 7, 2025 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants